-
Notifications
You must be signed in to change notification settings - Fork 4
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fixing caption numbering and positioning on Rmd report #157
base: main
Are you sure you want to change the base?
Conversation
…`, to move figure captions above each image
- Added `, provide=*` entry to `usepackage` `babel` - Added option to put caption on top for tables (required after all other changes) - Changed `_` to `-` on all chunk ids - Added
floatrow
to the skeleton.Rmd
header-includes
The connection errors that may appear on the checks should be fixed with PR #152 |
@GeraldineGomez these fixes work for me on Linux. I guess you should test it in Windows to ensure the reports are generated correctly |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi @jpavlich, thank you very much for collaborating on those changes and issues 🚀🙏😊.
Regarding Issue 159: The addition of bookdown works well on Windows.
I conducted some tests. When I generated the report in HTML format, the cover image was missing and displays the following text:
and the references in the HTML file appear like this:
The PDF file's cover displays the following text:
This might be due to the addition in the YAML of:
I believe this could be a good solution to address the warning related to the title when generating HTML. It's probable that some updates are necessary in the skeleton configuration.
…y-- work with rstudio properly
@GeraldineGomez Maybe the reason for the above behavior is that you selected the I guess I will have to thoroughly document that before release |
Fixes #156
Fixes #158
Fixes #166
Issue #159 may be fixed, although further testing is required